-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove the need for valid SSH keys linked to your github when using opam-publish #154
Conversation
Thanks. I downloaded the master of opam-publish and substituted your new publishSubmit.ml. Then I ran It still doesn't work - same error as before. Can you suggest a way to check that the new opam publish plugin was actually installed, as a first step? |
what did it fail with? Does it work with the latest commit?
opam plugins are simple executables with the The way |
oops i forgot to push said commit. It's done now. |
Sadly, it has gone from bad to worse, though not your fault. I can't even build
I have OpenSSL 3.1.2 installed, so I have no idea where dune is picking up I started with a fresh checkout of |
Does |
@johnwhitington i deleted your comment due to it containing an Anyway, this seems to show that this PR is quite dangerous and does not work. I'll close this then, sorry for the bother. Let's continue the discussion in #155 (the original opam issue is not the right place as opam-publish is a separate plugin) |
Thanks. Github auto-revoked the key, happily! |
Fixes ocaml/opam#5620
cc @johnwhitington if you haven't added SSH keys to your profile yet, could you try if this fixes your issue?